Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Prometheus monitoring with Grafana dashboard #48

Merged
merged 6 commits into from
Apr 13, 2022

Conversation

jardon
Copy link
Contributor

@jardon jardon commented Mar 25, 2022

Add prometheus and grafana relation for metrics scraping and visualization.

  • Prometheus relation
  • Grafana relation
  • Include Grafana dashboard

This PR cannot be merged until ops v1.4 is released as there is a bug that affects the storage.

Skipped config hash unit tests as they are not compatible with ops 1.4
An issue is created to keep track of this problem.

@jardon jardon requested a review from a team as a code owner March 25, 2022 14:17
@jardon jardon requested a review from a team as a code owner April 8, 2022 15:10
@ca-scribner ca-scribner merged commit 37e5980 into canonical:master Apr 13, 2022
jardon added a commit to jardon/minio-operator that referenced this pull request Jun 15, 2022
* Add Prometheus monitoring with Grafana dashboard (canonical#48)

* Add prometheus monitoring relation
* temporarily disable test that uses deprecated feature of ops library

Co-authored-by: Natasha Ho <[email protected]>

* Add charming actions (canonical#59)

* add charming actions

* update libs

* Add condition to only run publish action if repo owner is canonical

* Add condition to existing if

* Add condition to check-libs

* Move lib-check to merged condition

* Fix typo in condition

* Remove duplicate check-lib

* Add lib-check as prereq for publish-charm

* Remove publish-charm conditions from incorrect merge conflict

* test

Co-authored-by: Natasha Ho <[email protected]>
Co-authored-by: natalian98 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants